Skip to content

fix(codeowners): substitute the un-instantiated {{OWNER}} placeholder - #44

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/codeowners-substitute-owner
Jul 29, 2026
Merged

fix(codeowners): substitute the un-instantiated {{OWNER}} placeholder#44
hyperpolymath merged 1 commit into
mainfrom
fix/codeowners-substitute-owner

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

CODEOWNERS carried the literal @{{OWNER}}. GitHub's own validator reports every such rule as Invalid owner, so the file assigns no code owners at all — and any ruleset with require_code_owner_review becomes unsatisfiable.

just init does a repo-wide brace-token substitution and simply never ran here. Measured estate-wide: 54 repos carried it; 2 are genuine templates where the placeholder is correct and are deliberately untouched.

🤖 Generated with Claude Code

CODEOWNERS carried the literal `@{{OWNER}}`. GitHub's own validator
(/repos/{owner}/{repo}/codeowners/errors) reports every such rule as
'Invalid owner', which means the file assigns NO code owners at all -- and any
ruleset with require_code_owner_review becomes unsatisfiable, since a review
from a non-existent owner can never arrive.

This is not a template artefact here: `just init` performs a repo-wide brace-token
substitution and simply never ran for this repo. Measured estate-wide 2026-07-29:
54 repos carried the placeholder, of which 2 are genuine templates where it is
CORRECT and are deliberately untouched.

Substituted with the repo's actual owner (hyperpolymath).

Co-Authored-By: Claude Opus 5 <[email protected]>
@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ✅ Approved

Replaces the un-instantiated {{OWNER}} placeholder in the CODEOWNERS file with the correct team handle. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Jul 29, 2026

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@hyperpolymath
hyperpolymath merged commit 421b06e into main Jul 29, 2026
20 of 23 checks passed
@hyperpolymath
hyperpolymath deleted the fix/codeowners-substitute-owner branch July 29, 2026 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant